Skip to content

[stable33] refactor: flatten nesting with early returns#2579

Merged
miaulalala merged 1 commit intostable33from
backport/2564/stable33
May 7, 2026
Merged

[stable33] refactor: flatten nesting with early returns#2579
miaulalala merged 1 commit intostable33from
backport/2564/stable33

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented May 7, 2026

Backport of PR #2564

- Data::setOffsetFromSince(): invert guards with early returns and extract
  the repeated fallback fetch into getFirstKnownActivityHeader()
- MailQueueHandler::getAffectedUsers(): early return in the restrictEmails
  branch and extract the duplicated result loop into fetchAffectedUsers()

Signed-off-by: Anna Larch <anna@nextcloud.com>
AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@miaulalala miaulalala enabled auto-merge May 7, 2026 12:22
@miaulalala miaulalala merged commit 18d915d into stable33 May 7, 2026
55 checks passed
@miaulalala miaulalala deleted the backport/2564/stable33 branch May 7, 2026 14:43
@cypress
Copy link
Copy Markdown

cypress Bot commented May 7, 2026

Activity    Run #3726

Run Properties:  status check passed Passed #3726  •  git commit 8a758ee11c: [stable33] refactor: flatten nesting with early returns
Project Activity
Branch Review backport/2564/stable33
Run status status check passed Passed #3726
Run duration 03m 34s
Commit git commit 8a758ee11c: [stable33] refactor: flatten nesting with early returns
Committer backportbot[bot]
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 9
View all changes introduced in this branch ↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant